-
Notifications
You must be signed in to change notification settings - Fork 9
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: handle remaining translations #2478
Conversation
✅ Deploy Preview for regen-website ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
2f670c0
to
36faa55
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM pre-approving just a couple comments
@@ -4,19 +4,37 @@ import { OrderSummaryImage } from './OrderSummaryCard.Image'; | |||
import { OrderSummaryProps } from './OrderSummaryCard.types'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm actually moving this component to web-marketplace in another PR that hasn't been merged yet, but if that's too much work to revert the changes here, all good, I'll deal with the conflicts
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok, well, reverting this file would also revert some others. I think applying these changes to the moved file might be easier. Let me know if you need any help with that.
@@ -12,4 +14,6 @@ export const bottomTextMapping = { | |||
countryLabel: 'Country', | |||
stateLabel: 'State / Region', | |||
postalCodeLabel: 'Postal Code', | |||
locationStatePlaceholderLabel: 'State / Region', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
what's the difference between this and stateLabel
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The mock is incorrect; it should be Please choose a state,
I'll update it thanks.
Description
Handle remaining translations.
Author Checklist
All items are required. Please add a note to the item if the item is not applicable and
please add links to any relevant follow up issues.
I have...
How to test
Reviewers Checklist
All items are required. Please add a note if the item is not applicable and please add
your handle next to the items reviewed if you only reviewed selected items.
I have...